Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updates "btcli w set-identity" #146

Open
wants to merge 11 commits into
base: staging
Choose a base branch
from
Open

Updates "btcli w set-identity" #146

wants to merge 11 commits into from
+236 −71

Conversation

ibraheem-opentensor
Copy link
Contributor

  • Prompt the user for all fields (including sample for pgp)
  • Display all fields are optional
  • Explicitly asks which identity to update
  • Adds checks for each identity update use-case
  • Moves get_hotkey_owner to subtensor_interface

thewhaleking
thewhaleking approved these changes Oct 2, 2024
View reviewed changes
bittensor_cli/cli.py Outdated Show resolved Hide resolved
thewhaleking and others added 4 commits October 2, 2024 09:55
@thewhaleking
Retry Prompt Logic (#151)
dab8b02 
* Added retry prompt for use in set-id to allow for checking the input as it's entered for rejection/acceptance later.

* Updated helper text in set_id_prompts

* Use retry_prompt in set_id_prompts
@ibraheem-opentensor
Makes pgp and twitter fields optional
4e91df6
@thewhaleking
Fixed indentation issue
239e103
@ibraheem-opentensor
Handles edge cases and removes prompt for is_validator in nominate + …
7f12964 
…subnet create
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thewhaleking thewhaleking thewhaleking approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ibraheem-opentensor @thewhaleking